-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Titles sticky to menubar instead of document top #9584
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Titles sticky to menubar instead of document top #9584
Conversation
|
Confirmed: BurningTreeC has already signed the Contributor License Agreement (see contributing.md) |
✅ Change Note StatusAll change notes are properly formatted and validated! 📝 $:/changenotes/5.4.0/#9584Type: enhancement | Category: usability
🔗 #9584 👥 Contributors: BurningTreeC 📖 Change Note GuidelinesChange notes help track and communicate changes effectively. See the full documentation for details. |
✅ Deploy Preview for tiddlywiki-previews ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See code comment. Otherwise it seems to work as advertised. Tested with FF Ubuntu 24.04
Refactor menubarObserver variable declaration to be local within the startup function.
|
@BurningTreeC we should be able to use the mediaquery tracker/background actions from #8702 to simplify the approach. |
By popular demand I try adding a startup module for the menubar plugin that tracks the height of the menubar and sets the css variable --tc-menubar-height that gets applied to the sticky titles so that they stick to the menubar and are not hidden by it